Skip to content

Update upserting.md#211

Merged
szarnyasg merged 2 commits intoduckdb:mainfrom
johangithub:patch-2
Oct 13, 2025
Merged

Update upserting.md#211
szarnyasg merged 2 commits intoduckdb:mainfrom
johangithub:patch-2

Conversation

@johangithub
Copy link
Contributor

@johangithub johangithub commented Oct 12, 2025

  1. Insert was done with John, but rest of the example uses Jhon. John is more common way of spelling, but it's possible that the author intended to show how update changes the entire column.
  2. Anna's salary was never updated, so it should stay at 100_000, which then breaks the 2nd to last example, unless we change the merge condition to >=.
  3. One dangling comma
Screenshot 2025-10-11 at 6 28 20 PM Screenshot 2025-10-11 at 6 28 54 PM Screenshot 2025-10-11 at 6 30 16 PM

@szarnyasg

@szarnyasg
Copy link
Collaborator

@johangithub this is great, thanks a lot for the PR and the detailed explanation of the changes!

@szarnyasg szarnyasg merged commit 24260ea into duckdb:main Oct 13, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants